Skip to content

feat: add CLAUDE.md#17

Merged
pthmas merged 1 commit intomainfrom
pierrick/claude-md
Feb 25, 2026
Merged

feat: add CLAUDE.md#17
pthmas merged 1 commit intomainfrom
pierrick/claude-md

Conversation

@pthmas
Copy link
Collaborator

@pthmas pthmas commented Feb 25, 2026

Overview

Add CLAUDE.md to repo

Summary by CodeRabbit

  • Documentation
    • Added comprehensive architectural guide documenting system design, configuration, and operational best practices
    • Enhanced API documentation for /api/status endpoint with expanded response data and new /api/height endpoint

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

📝 Walkthrough

Walkthrough

Introduces and updates documentation for the Atlas EVM blockchain explorer, including a comprehensive architecture reference (CLAUDE.md) covering design decisions, operational guidance, and coding conventions, alongside API endpoint documentation updates describing the /api/status and /api/height endpoints with full chain status details.

Changes

Cohort / File(s) Summary
Documentation
CLAUDE.md, docs/API.md
New architectural reference document consolidating design decisions, environment configuration, and operational guidance; updated API documentation for /api/status and /api/height endpoints with detailed chain status response schema.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • smaller README #6 — Modifies docs/API.md for API endpoint documentation updates, with similar documentation scope and API reference focus.

Suggested reviewers

  • tac0turtle

Poem

🐰 A rabbit hops through docs so bright,
Atlas explorer, shining light!
From CLAUDE's guide to API's call,
Architecture documented for all! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: add CLAUDE.md' directly matches the main change—adding a new CLAUDE.md file. However, it omits that this file contains significant architectural documentation for Atlas, making the title somewhat incomplete in conveying the actual scope and value of the change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pierrick/claude-md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (1)
docs/API.md (1)

32-47: Consider adding a /api/height response example for consistency.

The /api/status response body is documented with a JSON example, but /api/height — described as the frequently-polled endpoint — has no corresponding example. Since CLAUDE.md (line 82) documents its shape as { block_height, indexed_at }, adding it here keeps the reference self-contained.

✏️ Proposed addition
 | GET | `/api/height` | Current block height and indexer timestamp (lightweight, safe to poll frequently) |
 | GET | `/api/status` | Full chain status: chain ID, chain name, block height, total transactions, total addresses |
 | GET | `/health` | Health check (returns "OK") |
 
+**`/api/height` response:**
+```json
+{
+  "block_height": 1000000,
+  "indexed_at": "2026-01-01T00:00:00+00:00"
+}
+```
+
 **`/api/status` response:**
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/API.md` around lines 32 - 47, Add a concrete JSON example for the
`/api/height` endpoint to match the documented shape in CLAUDE.md: insert a
small code block near the `/api/height` line showing `{ "block_height": 1000000,
"indexed_at": "2026-01-01T00:00:00+00:00" }`; ensure formatting is consistent
with the existing `/api/status` example and that the field names `block_height`
and `indexed_at` exactly match the names referenced in the docs.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@CLAUDE.md`:
- Line 3: Replace the unhyphenated phrase "ev-node based" with "ev-node-based"
in the sentence "Atlas is an EVM blockchain explorer (indexer + API + frontend)
for ev-node based chains." and add the language tag "text" to the directory-tree
fenced code block (the block beginning with "```" that contains the atlas/ tree)
so the fence becomes "```text" to satisfy MD040.

---

Nitpick comments:
In `@docs/API.md`:
- Around line 32-47: Add a concrete JSON example for the `/api/height` endpoint
to match the documented shape in CLAUDE.md: insert a small code block near the
`/api/height` line showing `{ "block_height": 1000000, "indexed_at":
"2026-01-01T00:00:00+00:00" }`; ensure formatting is consistent with the
existing `/api/status` example and that the field names `block_height` and
`indexed_at` exactly match the names referenced in the docs.

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 91b4de8 and 6e95c4a.

📒 Files selected for processing (2)
  • CLAUDE.md
  • docs/API.md

@@ -0,0 +1,126 @@
# Atlas — Claude Code Context

Atlas is an EVM blockchain explorer (indexer + API + frontend) for ev-node based chains.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Minor grammar and fenced-code-block nit.

Two small documentation quality issues flagged by the linters:

  1. Line 3ev-node based should be hyphenated: ev-node-based chains.
  2. Line 17 — The directory-tree fenced block has no language tag, triggering MD040. Adding text suppresses the warning.
✏️ Proposed fix
-Atlas is an EVM blockchain explorer (indexer + API + frontend) for ev-node based chains.
+Atlas is an EVM blockchain explorer (indexer + API + frontend) for ev-node-based chains.
-```
+```text
 atlas/
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Atlas is an EVM blockchain explorer (indexer + API + frontend) for ev-node based chains.
Atlas is an EVM blockchain explorer (indexer + API + frontend) for ev-node-based chains.
🧰 Tools
🪛 LanguageTool

[grammar] ~3-~3: Use a hyphen to join words.
Context: ...r (indexer + API + frontend) for ev-node based chains. ## Tech Stack | Layer | ...

(QB_NEW_EN_HYPHEN)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@CLAUDE.md` at line 3, Replace the unhyphenated phrase "ev-node based" with
"ev-node-based" in the sentence "Atlas is an EVM blockchain explorer (indexer +
API + frontend) for ev-node based chains." and add the language tag "text" to
the directory-tree fenced code block (the block beginning with "```" that
contains the atlas/ tree) so the fence becomes "```text" to satisfy MD040.

@pthmas pthmas merged commit a488e55 into main Feb 25, 2026
3 checks passed
@pthmas pthmas deleted the pierrick/claude-md branch February 25, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant